css node declaration: Avoid optional out parameters
authorMatthias Clasen <mclasen@redhat.com>
Fri, 11 Sep 2015 02:04:31 +0000 (22:04 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Sat, 12 Sep 2015 16:50:37 +0000 (12:50 -0400)
Always pass a pos to find_class, and avoid the if.

gtk/gtkcssnodedeclaration.c

index 87030d0f277cadd6b388949323ff80b0d75d5adb..da1c944bd357c581c79df8bc981f3bd1ac4b61d7 100644 (file)
@@ -231,8 +231,7 @@ find_class (const GtkCssNodeDeclaration *decl,
   GQuark *classes;
   guint pos;
 
-  if (position)
-    *position = 0;
+  *position = 0;
 
   if (decl->n_classes == 0)
     return FALSE;
@@ -264,8 +263,7 @@ find_class (const GtkCssNodeDeclaration *decl,
     }
   while (min <= max);
 
-  if (position)
-    *position = pos;
+  *position = pos;
 
   return found;
 }
@@ -326,6 +324,7 @@ gboolean
 gtk_css_node_declaration_has_class (const GtkCssNodeDeclaration *decl,
                                     GQuark                       class_quark)
 {
+  guint pos;
   GQuark *classes = get_classes (decl);
 
   switch (decl->n_classes)
@@ -346,7 +345,7 @@ gtk_css_node_declaration_has_class (const GtkCssNodeDeclaration *decl,
       return FALSE;
 
     default:
-      return find_class (decl, class_quark, NULL);
+      return find_class (decl, class_quark, &pos);
     }
 }